Skip to content

prettify CI output#2740

Closed
madrob wants to merge 1 commit intoml-explore:mainfrom
madrob:pretty-ci
Closed

prettify CI output#2740
madrob wants to merge 1 commit intoml-explore:mainfrom
madrob:pretty-ci

Conversation

@madrob
Copy link
Copy Markdown
Contributor

@madrob madrob commented Nov 6, 2025

Proposed changes

Use workflow commands to group log lines and make output easier to read.
https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-commands#grouping-log-lines

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING document
  • I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the necessary documentation (if needed)

@madrob madrob requested review from angeloskath and awni November 6, 2025 17:33
@zcbenz
Copy link
Copy Markdown
Collaborator

zcbenz commented Nov 6, 2025

@madrob Is it possible to split the single step into multiple steps in Actions? i.e. Instead of seeing a single build-cuda step that takes 44m, make the Actions show time spent for each step.

Screenshot 2025-11-07 at 8 28 06

It would make things easier to debug if something went wrong. For example we used to have a problem that a single test took 30m to run in CI, with current UI it would be much harder to notice the problem.

@madrob
Copy link
Copy Markdown
Contributor Author

madrob commented Nov 7, 2025

Yea, there's an open GitHub feature request to make each sub-step in a composite action have it's own timing information and header, but that's been around for a while and doesn't look like it will get any traction. We'd have to split things in the primary workflow which is totally doable but would require somebody with more knowledge of how MLX works to do a better job than I would, I think. Maybe just build and test make sense as the division for now. I don't plan to make these changes, but they would definitely be appreciated!

@awni
Copy link
Copy Markdown
Member

awni commented Nov 12, 2025

This needs a rebase.

@awni
Copy link
Copy Markdown
Member

awni commented Nov 18, 2025

Closing this since it's quite dated / getting incorporated into subsequent PRs.

@awni awni closed this Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants